Skip to content

AB#33315 align AI analysis tab height#2526

Merged
JamesPasta merged 1 commit into
devfrom
feature/AB#33315-align-ai-analysis-tab-height
Jun 3, 2026
Merged

AB#33315 align AI analysis tab height#2526
JamesPasta merged 1 commit into
devfrom
feature/AB#33315-align-ai-analysis-tab-height

Conversation

@jacobwillsmith
Copy link
Copy Markdown
Contributor

@jacobwillsmith jacobwillsmith commented Jun 3, 2026

Summary

  • Aligns the AI Analysis tab with the shared Grant Application tab styling.
  • Removes stale AI-specific tab CSS.

Validation

  • Manually verified on the Grant Application details page.
  • Build passed locally.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 3, 2026

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 752
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Grant Application details page tab styling so the AI Analysis tab matches the shared tab height/centering used across the details navigation, while removing AI-tab-specific CSS that’s no longer needed.

Changes:

  • Standardized .details-nav .nav-link layout to a consistent fixed height with centered content.
  • Added a small icon-specific adjustment (.details-nav .nav-link i) to keep icon vertical alignment consistent.
  • Removed unused AI-tab-specific styles for #ai-analysis-tab and related indicator/wrapper classes.

@JamesPasta JamesPasta merged commit 7bfb8d3 into dev Jun 3, 2026
26 checks passed
@JamesPasta JamesPasta deleted the feature/AB#33315-align-ai-analysis-tab-height branch June 3, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants